chore: verify versions #182
Conversation
|
Il va falloir investiguer pourquoi on génère deux fois la même version |
| @@ -14,6 +17,7 @@ export function _readAndFind(pattern: RegExp, versions: Map<string, string>) { | |||
| } | |||
|
|
|||
| versions.set(file, version); | |||
| consola.trace(`Found ${file.split("/").pop()}@${version}`); | |||
| consola.log(`Found ${file.split("/").pop()}@${version}`); | |||
There was a problem hiding this comment.
C'est ok pour moi de le passer en log.
.github/workflows/publish-edge.yml
Outdated
| @@ -26,7 +26,7 @@ jobs: | |||
| uses: codacy/[email protected] | |||
| with: | |||
| minor-identifier: "feat:" | |||
| release-branch: release | |||
| release-branch: main | |||
There was a problem hiding this comment.
pas convaincu de cette solution, ça fera une release sur main (3.6.0, par example) alors qu'on veut une snapshot (3.6.0-snapshot)
There was a problem hiding this comment.
Yes, mais dans ce cas, ça veut dire qu'il faut que release soit en avant sur main sauf qu'on ne veut pas à avoir à gérer release
There was a problem hiding this comment.
Je vais voir si on ne peut pas trouver un autre outil
There was a problem hiding this comment.
je retire ces modifs et je ferais une autre pr au besoin
There was a problem hiding this comment.
je ne sais pas pourquoi on est dans ce cas. Le truc qui a "débloqué" c'est que tu as fait un nouveau commit donc une nouvelle version. C'est pour ça que je ne suis pas convaincu de cette solution. Je me dis qu'on pourrait avoir le pb avec d'autres outils aussi
esoubiran-aneo
force-pushed
the
chore-fix-verify-version
branch
from
05b1112 to
17442a8
Compare
fix #183